ComboBox: Remove pointless signal ID member fields

We don’t need to store these for disconnection, as we can just use
g_signal_handlers_disconnect_by_data().

Update Slovak translation

ScrolledWindow: Fix non-disconnection of handlers

• Use disconnect_by_data() to catch both _adjustment_changed() and now
  _adjustment_value_changed(), as the latter had been missed until now.

• Also disconnect from indicator_value_changed(), which was not done in
  destroy() due to indicator_reset() and remove_indicator() disagreeing.

https://bugzilla.gnome.org/show_bug.cgi?id=775074

Window: Do not connect to NULL screen in init()

Do not connect to get_settings_for_screen() if we have no screen…

Use g_signal_connect(), not connect_object(), to match how set_screen()
makes this same connection, and how finalize() already disconnects it.

https://bugzilla.gnome.org/show_bug.cgi?id=705640

FileChooserWidget: Remove an unused #include

gskenums: Fix typo/grammar in GskRenderNodeType doc

Avoid computing the same thing twice

Trivial cleanup.

Entry: Rename int helper to gtk_entry_clear_icon()

It was called gtk_entry_clear(), which was unnecessarily vague.

Update Hungarian translation

Update Hungarian translation

Entry: Fix Shift-click → extend/truncate selection

Since the move from button-press to gesture events, Shift-clicking did
not work to start a selection (from none) or truncate an existing one.

This was due to the code being copy-pasted around and some logic being
broken in the process. This makes both of those work as they should, by
shuffling it again so the end result is the same as before. Highlights:

(1) ::button-press if extending due to a single press would call
set_positions(tmp_pos, tmp_pos), which is what made the Shift+click to
create a selection work. That was lost. Add it back to make that work.

(2) ::button-press in the “Truncate current selection” branch would not
execute all the stuff around “extend_to_left”, as that was the else
case. So, set extend_selection = FALSE so we skip over that later on.

(3) BUT! This Truncate case never fired because it was in the else
branch of if (in_selection())! Of course, it must be in the true branch.

(4) The IM context was not reset if the Shift-click occurred within an
existing selection, only if it did not. In ::button-press this was the
first thing done if extending a selection, regardless. Make it so again.

https://bugzilla.gnome.org/show_bug.cgi?id=780750

gl.po: Remove mnemonic underlines from tooltips

It looks like these were copy-paste errors from the corresponding labels
for buttons, which should have mnemonics. Of course, tooltips cannot.

Avoid empty space in the emoji chooser

We need to hide the empty flow boxes as well, to avoid
spacing between them.

https://bugzilla.gnome.org/show_bug.cgi?id=786966

Update Friulian translation

textview: fix bug on DnD displaced limits of selection

The fix of commit f2fd655754407103f8fb9b2c3e7586fb595ab917
should be confined to DnD coords only, because otherwise
it causes the start and end of the selection to be displaced.

https://bugzilla.gnome.org/show_bug.cgi?id=785736

GtkBuilder: Prefer "type-func" over "class" when looking for the GType

https://bugzilla.gnome.org/show_bug.cgi?id=786932

Update Serbian Latin translation

Update Serbian translation

GskRenderNode: Fix fallback border drawing

Update Latvian translation

Update Turkish translation

LevelBar: Really fix underallocation of blocks

Themes should not enforce min sizes on blocks in continuous mode; in
this case, the filled block should be as large as it needs to be to
reflect the current value, and no larger or smaller than that. So, the
fact that the minimal size was selected on just levelbar block is wrong:
we should also require the levelbar.discrete class to apply min sizes.

The widget should enforce whatever correct minimum size results from the
above fix, by reapplying commit 78b4885fe8850e132d8bb06df8ab90ac6c2033e0

Except: we should not allocate/draw the filled block if the value is 0,
as in this case, the LevelBar should be empty, not have a min-size fill.

This partially reverts commit 96062ffeae5245fa165a96a2af86d5645f5e8569,
as it makes sense to set min sizes for discrete blocks, so keep that in.

https://bugzilla.gnome.org/show_bug.cgi?id=783649

gtk/gtkshow: Remove deprected gtk_show_uri()

Use gtk_show_uri_on_window() instead gtk_show_uri()

Entry: Handle :show-emoji-icon becoming false

Disconnect the now-unwanted signal handler, and hide the icon.

https://bugzilla.gnome.org/show_bug.cgi?id=786940

Entry: Drop redundant typecasts

https://bugzilla.gnome.org/show_bug.cgi?id=786940

Entry: Only open emoji picker on 2ndary icon click

We hijack the secondary icon for the emoji picker, but the handler for
::icon-press did not check the pressed icon and opened it for either.

https://bugzilla.gnome.org/show_bug.cgi?id=786938

Updated Danish translation

Updated Danish translation

Updated Slovenian translation

Updated Slovenian translation

gskrendernode: Fix inset shadow drawing

gskrendernode: Constify some parameters

Revert "headerbar: Avoid allocating 0 visible children"

This reverts commit 8c0e5adaab3f38de31d44dd50bcc8179b17c8b4a.

This is actually needed since GtkHeaderBar will allocate and snapshot
widget that coun_visible_children does not consider.

Update Friulian translation

range: Fix trough state update

popover: Initialize some locals

gtk_popover_get_pointing_to does not fill the given rect in every case.

headerbar: Avoid allocating 0 visible children

Ths avoids doing a bunch of work as well as passing 0 to g_alloca which
is undefined.

Update Korean translation

Update Korean translation

Update Turkish translation

meson: Make sure ENABLE_NLS is actually defined

Despite the comment, we ended up without ENABLE_NLS.

x11: Fix managed dnd

We need to actually trigger the drop from the gdk side.

Remove an unused field

Drag contexts are objects, so there is no need to carry a
manual refcount around.

x11: Fix initial drag cursors

Under X, we were not setting the right drag cursor initially,
because at current_action == action == 0, initially. Fix this
by explicitly using the right cursor when grabbing.

wayland: Stop using subsurfaces for popovers

Subsurfaces don't currently work with our new rendering,
and this makes popovers unusable. We can go back to using
subsurfaces for popovers when this is fixed.

container: Remove leftover prototype

Thanks to Murray Cumming.

Update Friulian translation

Fix Catalan translation

Update Friulian translation

ComboBox: Clarify documentation of get_active_iter

https://bugzilla.gnome.org/show_bug.cgi?id=618160

Popover: Include window shadows in overshoot calcs

.update_position() enforces that non-Wayland platforms must position a
Popover within its parent Window. We use the allocation of the Window
to translate the position and check for overshoot on each of its sides.
Calling Widget.get_allocation() of a CSD Window includes its shadows.

But shadows were not excluded from the area in which we can position.
Thus, Popovers could get positioned in the shadow of CSD windows, where,
at least on X11, no input is received. Therefore, positioning a Popover
over a shadow meant its child widgets within that area became unusable.

Fix by calling Window.get_shadow() and including it in the overshoot on
each side. This adjusts for how the allocation includes shadows, making
overshoots with and without shadows the same. Thus, we avoid considering
shadows as viable for positioning, favouring a side where input works.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

testpopover: Use HeaderBar to get CSD decorations

This helps test whether the Popover positioning gets messed up by the
presence of CSD shadow or other accessories around the content area.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

Update Chinese (Taiwan) translation

Update POTFILES.in and POTFILES.skip

Remove motif dnd leftovers

We don't support Motif DND anymore, so no need to look for
Motif-specific messages.

SidebarRow: Add Unmount tooltip to eject_button

It was reported that the lack of a tooltip made its purpose unclear.
This can be solved by just copying PlacesViewRow’s eject_button tooltip.

https://bugzilla.gnome.org/show_bug.cgi?id=766909

fontchooser: Block row deleted signal handler when reloading model

This prevents the load_fonts() function from switching to the "no fonts"
page and back when the model is reloaded. Given
GtkSettings::gtk-fontconfig-timestamp is 0 on Wayland and style changes
happen often, the stack change messes up popovers and pointer focus
on the fonts treeview and test entry.

GtkFileChooserNativeQuartz: restore parent focus after closing dialog

Tested for both modal and non-modal dialogs

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: improve support for file filters

Instead of using conditional compilation, use respondsToSelector to
check at runtime for setAccessoryViewDisclosed.

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: add support for Mountain Lion and Mavericks

https://bugzilla.gnome.org/show_bug.cgi?id=784723 introduced support for
native file chooser dialogs on macOS, but due to the use of generics in
the patch, there will be compilation errors on pre-Xcode 7 platforms,
such as Mountain Lion and Mavericks.

I strongly recommend to revert this patch when the oldest supported
macOS release is bumped to Yosemite (10.10).

https://bugzilla.gnome.org/show_bug.cgi?id=785306

Revert "wayland: fix filechooser crash"

This reverts commit bd033165e52469845ac6f69c4fa9ac9b897f9bcc.

The function already had an early exit, no need for this check.

wayland: fix filechooser crash

The opaque region of the window can be updated before we're requested
its creation on the compositor.

https://bugzilla.gnome.org/show_bug.cgi?id=786673

wayland: fix filechooser crash

The size of the window can be updated before we're requested its
creation on the compositor.

https://bugzilla.gnome.org/show_bug.cgi?id=786673

magnifier: Port to snapshot

Instead of gtk_widget_draw'in the inspected widget inside the
magnifier's ::draw handler, just create a new GtkSnapshot and snapshot
in its snapshot handler, similar to what GtkStack is doing.

Update Friulian translation

widget: Fix UPDATES debug rendering

Choose the right widget to add the debug updates to (the one with the
window) and the correct region (the one in that window's coordinates).

testgtk: Remove some invalid css

widget: Remove unused draw_internal prototype

gtk_widget_draw_internal is now only used inside gtkwidget.c, so remove
the prototype from gtkwidgetprivate.h. And since all incovacations call
it with clip_to_size=TRUE, remove that parameter.

Update Italian translation

Update Italian translation

Updated Lithuanian translation

getting_started: Sync filenames & section headings

The order of Examples 3 and 4 was swapped, so the headings updated
themselves, but the filenames were left reflecting the old order.

https://bugzilla.gnome.org/show_bug.cgi?id=786553

Handle emoji data change in emoji chooser

Update the settings schema to follow the change in Emoji data,
and make the emoji chooser code handle the new format.

Add shortnames to emoji data

The json file is from

https://github.com/emojione/emojione/tree/master/extras/alpha-codes

It is under the MIT license.

Don't consume too many key releases

The Emoji input support in GtkIMContextSimple was
unintentionally consuming too many key release
events. Fix this.

https://bugzilla.gnome.org/show_bug.cgi?id=786594

build: Use the appropriate symbol visibility annotation

We cargo-culted this from Autotools, but GCC on Windows supports the
same __declspec syntax as MSVC. The only difference is the additional
flag needed for GCC-like compilers.

build: Fix macOS linker compatibility flags

The linker on macOS does not support '=' in its command line; there's no
guarantee that we are using the correct compatibility versions compared
to the Autotools build, but for that we'll need to build GTK+ master on
macOS.

build: Allow using libepoxy as a subproject

Like we do for Graphene, we can build libepoxy as a sub-project of GTK+
in case we do not find the dependency installed on the system.

Update Spanish translation

container: Fix scrolling adjustment coordinates

container: Remove gtk_container_propagate_draw

It's superseded by gtk_widget_snapshot_child now.

textview: Draw children using gtk_widget_snapshot_child

Update Nepali translation

Update Nepali translation

Update Malayalam translation

Update German translation

Update Kazakh translation

Update French translation

Update French translation

Updated Serbian translation

Update German translation

widget: Remove outdated comment about widget coordinates

Not so odd anymore.

Remove gdk_event_get_user_data from public headers

And add it back in gdk-private.h

gdk: wayland: always set buffer scale on size update

We're only updating this when we have an EGL window. This means we
have the wrong scaling factor when using Vulkan.

https://bugzilla.gnome.org/show_bug.cgi?id=786492

gsk: vulkan: fix image uploading

The memory alignment requirements are different from the image layout.
We want the rowPitch to know where to upload the lines.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
https://bugzilla.gnome.org/show_bug.cgi?id=786485

Update Croatian translation

Update Croatian translation

gdkwindow: Avoid re-setting the opaque region if it doesn't change

This avoids, at least, needless chatter with the compositor and the X
server in X11's case.

https://bugzilla.gnome.org/show_bug.cgi?id=786469

Update Slovak translation